Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: NPG wallet cards policy #1880

Merged
merged 3 commits into from
Apr 3, 2024
Merged

fix: NPG wallet cards policy #1880

merged 3 commits into from
Apr 3, 2024

Conversation

pietro-tota
Copy link
Contributor

@pietro-tota pietro-tota commented Mar 29, 2024

List of changes

Fix notification wallet NPG policy to take into account paymentMethod instead of paymentCircuit field for evaluate CARD payment

Motivation and context

Fix notification wallet NPG policy to take into account paymentMethod instead of paymentCircuit field for evaluate CARD payment

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Does this introduce a change to production resources with possible user impact?

  • Yes, users may be impacted applying this change
  • No

Does this introduce an unwanted change on infrastructure? Check terraform plan execution result

  • Yes
  • No

Other information


If PR is partially applied, why? (reserved to mantainers)

@pietro-tota pietro-tota marked this pull request as ready for review March 29, 2024 18:13
@pietro-tota pietro-tota requested review from a team as code owners March 29, 2024 18:13
@pietro-tota pietro-tota added env: dev The status refers to DEV environment env: uat The status refers to UAT environment status: applied ✋ labels Apr 3, 2024
@pietro-tota pietro-tota merged commit 409f520 into main Apr 3, 2024
5 of 7 checks passed
@pietro-tota pietro-tota deleted the CHK-2636-fix-policy branch April 3, 2024 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
env: dev The status refers to DEV environment env: uat The status refers to UAT environment status: applied ✋
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants